Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement webauthn extra origins & Raise KeyCloak version to 24.0.5 in acceptance tests #313

Merged

Conversation

TuningYourCode
Copy link
Contributor

#312 implements quiet a few properties on the realm resource.

The webauthn & webauthn passwordless extra origins property require at least KeyCloak 23.0.0 as it was added with that release.

This PR raises the minimum KeyCloak version used for acceptance testing and will add these properties.

.github/workflows/ci.yaml Outdated Show resolved Hide resolved
@treydock
Copy link
Owner

There are now merge conflicts due to other PR getting merged.

Also this needs to be bumped to new minimum version:

let(:version) { '22.0.0' }

String $version = '22.0.0',

@treydock
Copy link
Owner

If the latest version is 24.x it might be better to just bump the minimum version to 24.x since it looks like Keycloak only supports n-1.

@treydock
Copy link
Owner

Actually, make 24.x the minimum as seems 25.0.0 is now released.

@TuningYourCode TuningYourCode force-pushed the implement-webauthn-extra-origins branch 3 times, most recently from 700cd41 to abcb4ac Compare June 20, 2024 10:35
@TuningYourCode TuningYourCode changed the title Implement webauthn extra origins & Raise KeyCloak version to 23.0.7 in acceptance tests Implement webauthn extra origins & Raise KeyCloak version to 24.0.5 in acceptance tests Jun 20, 2024
@treydock
Copy link
Owner

Looks like 25.0.0 may break several things. I think it'd be fine to just upgrade this to 24.0.5 as default and only tested version and then do 25.0.0 separate which I can help with.

@TuningYourCode TuningYourCode force-pushed the implement-webauthn-extra-origins branch from abcb4ac to 9c719fb Compare June 24, 2024 09:30
@TuningYourCode
Copy link
Contributor Author

Thanks, yeah looks like 25.0.0 has a few things which need changes in the module or at least in tests.

@treydock treydock merged commit 3b639d2 into treydock:master Jun 24, 2024
16 checks passed
@treydock
Copy link
Owner

I am working on 25.x support and so may not release this until I release the support for 25.x. There are some breaking changes to support 25.x that will also require a major version bump.

@TuningYourCode
Copy link
Contributor Author

No problem, i don't need these properties anyway - just implemented them to have all present in the module.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants